-
Notifications
You must be signed in to change notification settings - Fork 65
Remove OMCSessionZMQ #387
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Remove OMCSessionZMQ #387
Conversation
3a851c2 to
4b7dfa7
Compare
130a54b to
2918597
Compare
475cbbd to
93f2bd9
Compare
|
Rebase |
93f2bd9 to
9fa8f5e
Compare
|
This is marked as draft. I believe we are ready now to start merging,
But we should first update the users guide https://openmodelica.org/doc/OpenModelicaUsersGuide/latest/ompython.html |
I'm currently working on PR #404 - get Runner included / keep backward compatibility. The main points are:
The code defined in the RFC #404 updates the internal structure of OMPython such that:
Furthermore, as OMCPath is working only for Python >= 3.12, another dimension exists. There is a dummy implementation of OMCPath for Python < 3.12 such that most of the functionality can also be used here. See PR #401 for a hint that even older Python version are used (Python 3.10 or even lower). I did hold of documentation to finish work on this part; the new structure would be baseline of the documentation. My idea was to include the changes in 4.1.0 and create a cut at the next release (or keep some kind of compatibility layer alive). To keep best compatibility, I did set the listed PR's as draft, i.e. to be included later. |
finally remove OMCSessionZMQ
based on top of PR #386
milestone v5.0.0 (cleanup / removal of depreciated features)